feat: Node.js SDK update for version 25.1.0#153
Conversation
This reverts commit 72952ef.
Greptile SummaryThis PR bumps the Node.js SDK to version 25.1.0, adding new model fields, runtime enum entries, and correcting advisor documentation examples.
Confidence Score: 5/5This PR is safe to merge — all changes are additive model updates, enum extensions, and doc corrections with consistent test coverage. All changes are mechanical SDK updates tied to the 25.1.0 server release: a new required field on the File model (backed by updated fixtures), optional-ifying billing fields that the server no longer guarantees, three new Deno enum entries, corrected advisor auth examples, and version string bumps. Each change has matching test updates and nothing touches control flow or security-sensitive paths. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "feat: update Node.js SDK to 25.1.0" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| Deno121 = 'deno-1.21', | ||
| Deno124 = 'deno-1.24', | ||
| Deno135 = 'deno-1.35', |
There was a problem hiding this comment.
deno-1.21, deno-1.24, and deno-1.35 are added before the already-existing deno-1.40, deno-1.46, and deno-2.0 entries. Were these legacy runtimes newly enabled in the Appwrite 25.0.1 server release, or is this a backfill that was accidentally omitted from earlier SDK versions? Confirming intent helps reviewers of any downstream SDK parity checks.
| Deno121 = 'deno-1.21', | ||
| Deno124 = 'deno-1.24', | ||
| Deno135 = 'deno-1.35', |
There was a problem hiding this comment.
The Runtime enum mirrors the BuildRuntime change — deno-1.21, deno-1.24, deno-1.35 land before deno-1.40. If this is intentional (server 25.0.1 newly enables these legacy Deno targets), a brief note in the CHANGELOG entry would clarify why older versions are appearing in a minor bump.
* Added `sizeActual` property to file model for actual stored size after compression * Added `Deno121`, `Deno124`, and `Deno135` runtime options to `BuildRuntime` and `Runtime` enums * Updated advisor authentication examples to use API key instead of session * Updated billing limits properties to be optional in project models
premtsd-code
changed the title
ChiragAgg5k
left a comment
ChiragAgg5k
left a comment
There was a problem hiding this comment.
LGTM. Auto-generated SDK updates consistent with the matching PRs in the other language SDKs (sizeActual on File, BillingLimits optional, Deno 1.21/1.24/1.35 added, advisor docs switched to API key auth, version + changelog bump).
2 participants
This PR contains updates to the Node.js SDK for version 25.1.0.
Changes
sizeActualproperty to file model for actual stored size after compressionDeno121,Deno124, andDeno135runtime options toBuildRuntimeandRuntimeenums